Skip to content

Conversation

RalfJung
Copy link
Member

No need to pull in an external dependency if libstd already includes this feature (using the same dependency internally, but... still).

r? @oli-obk

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 30, 2020
@RalfJung
Copy link
Member Author

I checked in the code that std::backtrace lazily resolves symbols, but let's still make sure that this does not make anything slower.

@bors try @rust-timer queue

@rust-timer
Copy link
Collaborator

Awaiting bors try build completion

@bors
Copy link
Collaborator

bors commented Apr 30, 2020

⌛ Trying commit 30b32c6 with merge f1ecbbb3d28b7ebfb433e60f9a78a39ec8a6380a...

@bors
Copy link
Collaborator

bors commented Apr 30, 2020

☀️ Try build successful - checks-azure
Build commit: f1ecbbb3d28b7ebfb433e60f9a78a39ec8a6380a (f1ecbbb3d28b7ebfb433e60f9a78a39ec8a6380a)

@rust-timer
Copy link
Collaborator

Queued f1ecbbb3d28b7ebfb433e60f9a78a39ec8a6380a with parent eece58a, future comparison URL.

@rust-timer
Copy link
Collaborator

Finished benchmarking try commit f1ecbbb3d28b7ebfb433e60f9a78a39ec8a6380a, comparison URL.

@RalfJung
Copy link
Member Author

RalfJung commented May 1, 2020

Well, I'd say there is no measurable difference.

@oli-obk
Copy link
Contributor

oli-obk commented May 1, 2020

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented May 1, 2020

📌 Commit 30b32c6 has been approved by oli-obk

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels May 1, 2020
Dylan-DPC-zz pushed a commit to Dylan-DPC-zz/rust that referenced this pull request May 1, 2020
Miri: port error backtraces to std::backtrace

No need to pull in an external dependency if libstd already includes this feature (using the same dependency internally, but... still).

r? @oli-obk
Dylan-DPC-zz pushed a commit to Dylan-DPC-zz/rust that referenced this pull request May 1, 2020
Miri: port error backtraces to std::backtrace

No need to pull in an external dependency if libstd already includes this feature (using the same dependency internally, but... still).

r? @oli-obk
Dylan-DPC-zz pushed a commit to Dylan-DPC-zz/rust that referenced this pull request May 1, 2020
Miri: port error backtraces to std::backtrace

No need to pull in an external dependency if libstd already includes this feature (using the same dependency internally, but... still).

r? @oli-obk
bors added a commit to rust-lang-ci/rust that referenced this pull request May 2, 2020
Rollup of 6 pull requests

Successful merges:

 - rust-lang#71712 (Miri: port error backtraces to std::backtrace)
 - rust-lang#71736 (bootstrap: also apply unused-attributes hack without deny_warnings)
 - rust-lang#71738 (remove AllocId generalization of Pointer)
 - rust-lang#71739 (remove obsolete comment)
 - rust-lang#71781 (Uncomment test code for failure to use `Box::pin`)
 - rust-lang#71782 (Use a non-existent test path instead of clobbering /dev/null)

Failed merges:

r? @ghost
@bors bors merged commit cd824a5 into rust-lang:master May 2, 2020
@RalfJung RalfJung deleted the error-backtrace branch May 3, 2020 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants